Skip to content

fix(timeline): fix tickFormat being ignored#4463

Merged
GordonSmith merged 1 commit into
hpcc-systems:candidate-3.x.xfrom
jeclrsg:fix-eclwatch-timeline-formatting
Oct 16, 2025
Merged

fix(timeline): fix tickFormat being ignored#4463
GordonSmith merged 1 commit into
hpcc-systems:candidate-3.x.xfrom
jeclrsg:fix-eclwatch-timeline-formatting

Conversation

@jeclrsg
Copy link
Copy Markdown
Contributor

@jeclrsg jeclrsg commented Oct 13, 2025

Fixes an issue where tickFormat called by consumers of this component was being ignored / overwritten by _axisLabelFormatter. This resulted in d3-time-format defaulting back to it's 12-hour "AM/PM" notation

Checklist:

  • The commit message is properly formatted and free of typos.
    • The commit message title makes sense in a changelog, by itself.
    • The commit message includes a "fixes" reference if appropriate.
    • The commit is signed.
  • The change has been fully tested:
    • I have viewed all related gallery items
    • I have viewed all related dermatology items
  • There are no similar instances of the same problem that should be addressed
    • I have addressed them here
    • I have raised new issues to address them separately

Testing:

@jeclrsg jeclrsg requested a review from GordonSmith October 13, 2025 16:36
Comment thread packages/timeline/src/ReactTimeline.ts
Comment thread packages/timeline/src/ReactTimelineSeries.ts
Copy link
Copy Markdown
Member

@GordonSmith GordonSmith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

acutally it would be called "_reset" similar to "_exists"?

Fixes an issue where tickFormat called by consumers of this component
was being ignored / overwritten by _axisLabelFormatter. This resulted
in d3-time-format defaulting back to it's 12-hour "AM/PM" notation

Signed-off-by: Jeremy Clements <79224539+jeclrsg@users.noreply.github.com>
@jeclrsg jeclrsg force-pushed the fix-eclwatch-timeline-formatting branch from bda790a to d5d13fb Compare October 15, 2025 15:16
@jeclrsg jeclrsg requested a review from GordonSmith October 15, 2025 15:26
@GordonSmith GordonSmith merged commit 8d970df into hpcc-systems:candidate-3.x.x Oct 16, 2025
1 check passed
@jeclrsg jeclrsg deleted the fix-eclwatch-timeline-formatting branch April 24, 2026 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants